-
-
Notifications
You must be signed in to change notification settings - Fork 14.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add unrar license and use #341115
base: master
Are you sure you want to change the base?
add unrar license and use #341115
Conversation
It just means that the use restriction is incompatible with Fedora’s licence policy. I think it is still compatible with our lenient definition of |
The docs say that we would distribute it then https://github.com/SuperSandro2000/nixpkgs/blob/a1ddda46fa838b99460ae4594e3e382231ac1e8b/doc/stdenv/meta.chapter.md#liblicensesunfreeredistributable-unfree-redistributable-liblicensesunfreeredistributable-unfree-redistributable |
Thanks for picking this up! Thinking more on the discussion in #328698, I do think that there's actually a matryoshka of two licenses here:
Some projects have elected to include the code only along with the 2nd paragraph, which does not reference the rest of the license, so it wouldn't be quite correct to lump them together. Notably, the "2nd paragraph only" variation does look like it should allow redistribution without issues. |
Weird. The docs lie, I guess.
IIRC it’s actually that the licence changed to become more burdensome later on (otherwise any project cutting out parts of the licence would be committing a copyright violation and be less redistributable), but I forget the exact details. |
I think it is intentional. Some programs are unfree (as in open source free) but we are allowed to distribute them. |
We don’t do it. See #83884, and #83884 (comment) for why we shouldn’t do it as a blanket policy separate to any ideological concerns. |
Then we should update the docs to be correct. Since I seem to lack some knowledge here I leave it up to the people that are more familiar with licenses. |
a8c4ead
to
1009145
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m still not sure about redistributable = false;
here. The licence explicitly gives permission to redistribute and the use restrictions are no more onerous than some other licences we consider redistributable in the tree.
One formatting nit. Also, I believe there may be multiple versions of this licence, and I haven’t checked if the same version applies to all these packages.
else lib.licenses.gpl3Plus; | ||
license = [ lib.licenses.gpl3Plus ] | ||
# the extra unfree is because gpl license cannot legally be combined with unfree licenses | ||
++ lib.optionals unrarSupport [ lib.licenses.unrar lib.licenses.unfree]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++ lib.optionals unrarSupport [ lib.licenses.unrar lib.licenses.unfree]; | |
++ lib.optionals unrarSupport [ lib.licenses.unrar lib.licenses.unfree ]; |
Description of changes
Continuation of #328698
Based on the fedora comment I don't think the license allows us to redistribute the binaries.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.